Skip to content

chore: run zizmor on workflows and fix issues - #366

Merged
amoeba merged 2 commits into
mainfrom
chore/ci-zizmor
Apr 28, 2026
Merged

chore: run zizmor on workflows and fix issues#366
amoeba merged 2 commits into
mainfrom
chore/ci-zizmor

Conversation

@amoeba

@amoeba amoeba commented Apr 28, 2026

Copy link
Copy Markdown
Member

I ran zizmor . and had an agent work through the changes. The changes included,

  • SHA pinning
  • Removing template injection risks
  • Minimizing permissions and scoping to just needed
  • Removing some caching, leaving other caching in place since it's guarded reasonably well
  • Adding persist_credentials: false to actions/checkout steps
  • Switching on pull_request_target workflow to pull_request which runs with lesser permissions

@amoeba
amoeba merged commit 5e9eccd into main Apr 28, 2026
7 checks passed
@amoeba
amoeba deleted the chore/ci-zizmor branch April 28, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants